testdialog: Error out if templates are missing

Coverity complained about us ignoring the return value
here, and it is nicer to get an error message about this.

cups: Annotate a call whose return value we don't care about

We're closing this fd, but we're not going to do anything else
if that fails. Annotate to let coverity see this.

inspector: Check a return value

We don't care about the error here, but we were relying on
g_resources_get_info to zero the size even in case of error.
No need to do that, we can just check the return value. Plus,
it makes coverity happy.

inspector: Annotate a call whose return value we don't need

The way this code is written, we know that there is an item in
the hash table and the iter_next call will give it to us, so
no need to check the return value. Annotate the call to tell
coverity.

gtkicontheme: Check a return value

We were already looking at the error anyway, but rewriting things
this way lets coverity see the light.

gtkbuilderparser: Add some assertions

Add some assertions that things are non-NULL when we know they
are, so that coverity gets it.

gtk-demo: Check a return value

Found by coverity.

css: Fix GtkCssChange enum

Previously, the (1<<31) value evaluated to the 64bit value
  0xFFFFFFFF80000000
instead of
  0x0000000080000000
Avoid this by explicitly casting the value to unsigned long long.

Also cast all values to unsigned long long to achieve consistency
and hopefully in the future get new values added the same way.

headerbar: Switch css positions on direction change

Updated Hungarian translation

file chooser: Improve popover positioning

We keep the popover vertically tied to the selected row, but pick
up the horizontal position from the click / touch.

places sidebar: Preserve ::populate-popup functionality

We "fix" this signal the same way we did for GtkEntry and
GtkTextView: slightly change the signature so that the passed
widget may be a container other than a GtkMenu, and add a
property to explicitly opt in to receiving the signal in
this case.

file chooser: Use a popover for the context menu

Use a popover for the context menu, and add a long press gesture
to allow opening it with touch.

places sidebar: Use a popover for the context menu

Use a popover for the context menu, and add a long press gesture
to allow opening it with touch.

gdk: fix crash if visual is NULL

cairo_xlib_surface_get_visual might return NULL so we need to
check for NULL before trying to use it.

https://bugzilla.gnome.org/show_bug.cgi?id=752520

inspector: Avoid a gratitious use of g_strv_length

No need to count the strings first before iterating over them.

file chooser: Make bindings work in save entry

With the name entry being in the header bar now, we no longer
get bindings working by just letting the key event bubble up,
we have to explicitly apply them on key events that the save
entry is not handling.

GtkFileChooserButton: Don't update icon/label so often

They were updated in style-changed, causing the label to get set to
(None), then to the actual file name again a frame later, both of the
updates cause the GtkFileChooserButton to resize, possibly to the
minimal width, causing the layout to jump. Fix this by only updating
icon/label in style-updated when the icon theme actually changed, which
is the only case we care about here.

https://bugzilla.gnome.org/show_bug.cgi?id=752509

gtkprintunixdialog: Set border_width to 0

Avoid O(n²) walking of string arrays

"Yo, we heard you like traversing NULL-terminated arrays to operate on
them, so we called g_strv_length() as the for condition, so you can
iterate the array while iterating the array."

Instead of making famed rapper and television producer Xzibit proud, we
should avoid calling g_strv_length() on an array while looping on the
array, to avoid quadratic complexity.

We do this in various places that deal with arrays of strings that we
cannot really guess are short enough not to matter — e.g. the list of
CSS selectors in the inspector, or the required authentication
information for printing.

placessidebar: show persistent drives when not showing Other Locations

Previous commits changes the behavior of GtkPlacesSidebar, as it stops
handling persistent devices and only manages mounted networks, XDG
directories and bookmarks.

By radically breaking the previous behavior, we may have trouble
since we provide no alternatives to that besides the private widget
GtkPlacesView.

Fix that by showing the persistent devices when not showing Other
Locations item.

tests/gears: Ensure we don't use unavailable GL resources

We need to check on realize if we have access to a GL context, before
calling GL functions. We use gtk_gl_area_get_error() for that.

We also need to tear down the resources during unrealization, instead
of leaking them.

placessidebar: fix Connect to Server position

Commit 7db399d975b8f9626c21761dde5f2d5feeb6e305 introduces
the Other Locations item, as well as some behavioral changes
to GtkPlacesSidebar. It, howevers, wrongly changes the section
of code that handles the Connect to Server item, with various
side effects depending on the environment setup.

Fix that by adding the Connect to Server at the right section
of code.

Updated Hebrew po-properties translation

Updated Hebrew translation

Updated Hebrew translation

Updated Spanish translation

Updated Spanish translation

Updated POTFILES.in and POTFILES.skip

tests: Remove testplacesview test

The GtkPlacesView widget was made private, so we can't create it in a
testcase.

tests: Fix gcc complaints about testfilechooser

headerbar: Get rid of get_path_for_child()

Instead, reorder CSS nodes.

headerbar: Finish widget setup before showing it.

headerbar: forall() the button boxes in the right order

headerbar: Remove unused variables in private struct

toolbar: Switch from get_path_for_child() to css node managing

cssnode: Add gtk_css_node_reverse_children()

cssnode: Change API of some functions

gtk_css_node_set_after/before() are now called
gtk_css_node_insert_after/before().

This brings them in line with other similar APIs (ie GtkListStore). And
it allows easier usage of the API (see changes to gtkbox.c).

filechooserwidget: use places view to manage fixed devices

Previous patch modified places sidebar widget to stop handling
fixed devices by adding an "Other Locations..." item. Up to now,
however, these changes are isolated from each other since the
bundled file manager widgets ignore the sidebar requests for
external management of fixed devices and networks.

To fix that, make the file chooser widget be aware of the
GtkPlacesSidebar::show-other-locations signal and, when requested,
show places view to manage the fixed devices and networks.

https://bugzilla.gnome.org/show_bug.cgi?id=752034

placessidebar: add Other Locations item

Places sidebar is a widget that enabled the user to select
XDG directories, bookmarks and mounted network locations,
as well as manages permanent and removable devices.

The new design that aims to look less clutered makes the
sidebar display only removable devices, as well as mounted
networks, bookmarks and XDG directories, and delegates the
management of permanent devices such as hard drive partitions
to GtkPlacesView, a newly introduced widget for this specific
purpose.

To delegate it, add an "Other Locations..." item to notify
when the permanent devices manager is required. Besides that,
don't show these fixes devices on the sidebar itself, as they
are not supposed to be handled by the sidebar anymore.

https://bugzilla.gnome.org/show_bug.cgi?id=752034

placesview: add view for fixed drives and networks

Places sidebar shows XDG directories, mounted and unmounted devices,
connected networks, bookmarks and actions like 'Connect to server'
and 'Insert location', which causes the sidebar to grow very quickly
and look cluttered. Because of that, new mockups for the sidebar try
to simplify it.

To make the sidebar simpler, the new mockups propose that it should
only handle connected networks and removable devices such as flash
drives and USB devices, and delegates other devices for external
widgets through the 'Other Locations' item.

To handle fixed devices and manage network connections, add a new
widget named GtkPlacesView, based on Nautilus mockups to keep
consistency between GNOME file management tools - in this case,
between Nautilus and the bundled Gtk's file chooser.

https://bugzilla.gnome.org/show_bug.cgi?id=752034

Revert "x11: Query pointer devices' scroll valuators on toplevel enter events"

This reverts commit 77b8495bc4e51826afa451914d6a3b0df1b62a24.

The commit broke more scenarios than fixed, better to go back to square one.

GdkDisplayX11: Properly translate server timestamps from _NET_WM_FRAME_* messages

When using frame times from _NET_WM_FRAME_DRAWN and _NET_WM_FRAME_TIMINGS, we
were treating them as local monotonic times, but they are actually extended-precision
versions of the server time, and need to be translated to monotonic times in the
case where the X server and client aren't running on the same system.

This fixes rendering stalls when using X over a remote ssh connection.
https://bugzilla.gnome.org/show_bug.cgi?id=741800

places sidebar: Handle menu key

It is good practice to handle the menu key and Shift-F10 to
allow keyboard-driven use of context menus.

mir: don't ensure_surface if window not visible

mir: don't warn if transient_for is NULL

mir: add more default settings under mir

Updated Portuguese translation

Updated Spanish translation

file chooser: Show progess spinner even if not empty

With the previous approach, we could only show the spinner
before we had any results. With the new approach, we can just
leave the timeout in place and always show the spinner until
the search is done.

file chooser: Stop search when entry is cleared

file chooser: Refine search progress display

Move the spinner to the search bar, so we can keep it around
until the search is over.

file chooser: Be more careful with subtitles

We were showing things like "Searching in (null)" if the current
folder is not in the sidebar. Avoid that by falling back to using
current_folder.

Pointed out by Carlos Soriano.

GtkApplication: avoid using stale timestamps

Avoid using a stale timestamp (from the last user interaction with the
application) when a message arrives from D-Bus requesting that a new
window be created.

In this case the most-correct thing that we can do is to use no
timestamp at all.

We modify gdk_x11_display_set_startup_notification_id() to allow a NULL
value to mean "reset everything" and then call this function
unconditionally on receipt of D-Bus activation requests.  The result
will be that a missing desktop-startup-id in the platform-data struct
will reset the timestamp.

Under their default configuration metacity and mutter will both map
windows presented with no timestamp in the foreground.  This could
result in false-positive, but there is very little we can do about that
without the original timestamp from the user event.

https://bugzilla.gnome.org/show_bug.cgi?id=752000

gtkframe: Don't queue a redraw on the entire widget every size allocate

Lots of applications often use GtkFrame as a giant toplevel container,
and that means that they get size allocated often. When frames get size
allocated, they invalidate their entire widget tree, even if they
haven't changed size or anything like that happens at all. This is
because the shadow / label needs to be redrawn if the child changes
size. We can optimize this out and only mark ourselves for a redraw if
the child has actually changed its size.

GtkButtonBox: remove spacing when buttons are linked

GtkButtonBox adds the "linked" class to its style context when its
layout is set to GTK_BUTTONBOX_EXPAND. It shouldn't ever make sense to
have spacing between buttons in that case, as themes generally draw
linked elements with a continuous border.

Thus, always set spacing to 0 and ignore GtkDialog's button-spacing
style property when the layout is set to EXPAND.

Also remove the now-redundant css rules which set button-spacing to 0
for message dialogs.

https://bugzilla.gnome.org/show_bug.cgi?id=752131

range: Tweak button bindings

Change things around so that warp-to-click and jump-by-pages are
bound to left-click and shift-left-click, depending on the value
of gtk-primary-button-warps-slider. Autoscrolling is bound to
right-click.

To achieve this, reorganize gtk_range_multipress_gesture_pressed
so that the functions are clearly separated.

Updated Spanish translation

Updated Spanish translation

GtkSettings: Fix a documentation typo

range: Trivial field ordering

range: Add a stepper alternative

Make shift right-click on the trough start autoscrolling. The
autoscrolling is similar to the way steppers operate now, with
the difference that we vary the scrolling speed based on the
distance of the pointer from the widget.

range: switch secondary and middle button on steppers

Clicking on steppers does different things depending on which
button you use. We used to scroll to the end on secondary click
and use high-speed autoscrolling on middle-click. Switch these
two around, since the scroll-to-end functionality is less likely
to be useful, and the seconary button makes the autoscrolling
more easily available.

font chooser: Remove a stray notify

This was forgotten when I changed to font map to not be a
property.

file chooser: Clean up unused includes

file chooser: Remove some obsoltete code

We no longer have editable rows in the list, so no need to
check for them anymore.

file chooser: Fix multi-selection

Even if we only ever hit the code with a singleton selection,
calling gtk_tree_selection_get_selected is not ok if the tree
selection mode allows multi-selection. Replace all calls to
gtk_tree_selection_get_selected in the file chooser code with
callback loops iterating over the selection. This problem
was introduced with the recently added rename and delete
menuitems.

file chooser: Do less work

We only need to update the sensitivity of the context menu
items rihgt before we are going to pop it up. Everything else
is pointless work.

file chooser: Avoid doing excess work

Whenever we change directories, we unset the model, and then
we set a new model. This causes several emissions of
GtkTreeSelection::changed, for each of which we do a bunch
of work to update the path bar, the location entry, etc.

We can savely ignore some of these signals, and do less work.

Trivial code cleanup

file chooser: Avoid animations when setting up

There is no need to animate things when we are just setting up
the startup mode, so disable transitions in the revealer and
the stack. Pointed out by Carlos Soriano

file chooser: Ensure we update subtitles

Notify the ::subtitle property when modes change that may
affect it. Also ensure that we update the location mode when
we switch back to the pathbar.

cssnode: Only queue_validate() when node is visible

This avoids running animations in hidden toplevels.

cssnode: Only invalidate sibling nodes if changed node is visible

Updated Portuguese translation

Added an additional explanation and code snippet for GtkImageMenuItem.c

https://bugzilla.gnome.org/show_bug.cgi?id=752093

Added a more meaningful description for /deprecated/gtkimagemenuitem.c

https://bugzilla.gnome.org/show_bug.cgi?id=752088

places sidebar: Fix rows getting 'lost' when dnd fails

There was some code that lead to the places sidebar loosing track
of the row being dragged if a motion event came in between letting
go of the button and the drag snap-back animation ending. This would
cause us to not show the row again, giving the appearance that it
was 'lost'.

https://bugzilla.gnome.org/show_bug.cgi?id=751448

testtoolbar: Don't expand rows that shouldn't be expanded

frame: Remove get_path_for_child implementation

As far as I can read the code it added the .frame style class to the
GtkFrame element. But GtKFrame already has a .frame style class these
days.

actionbar: Remove get_path_for_child implementation

It wasn't used because CSS walks the widget->parent chain, so it'd only
ever call this func for priv->revealer.

css: text-decoration-style is not inherited

Simplify Xft setting fallback

If we don't find Xft values in the X resource db, simply fall
back to the values that are hardcoded in /etc/X11/Xresources
anyway. Extra trickery with likely-made-up screen dimensions
is not going to yield better results, and only makes for a
deeper rabbit hole when debugging.

gtksidebarrow: remove unused property

The sensitive property was a workaround which commit
3f8982a0cd7b779fe023bbd fixed.
Just remove it now that is not necessary.

gtkrevealer: fix preferred width calculation

A typo which was using minimum_width for natural_width and viceversa.

Fix a minor typo

Updated Polish translation

gdkwindow-quartz: partial aspect ratio support

Support was added for GDK_HINT_ASPECT in
gdk_quartz_window_set_geometry_hints though with one restriction:
min_aspect and max_aspect have to be equal, which I believe corresponds
to the most common usage. A warning will be printed if this condition is
not met but min_aspect will be used anyway.

testgmenu: use the menu binding code

The menu tracker does a better job of this than we can, so move over to
using it instead.

This fixes issues with './testgmenu --import' not properly displaying
the language submenu.

https://bugzilla.gnome.org/show_bug.cgi?id=752016

gtkdnd: Fix introspection annotations in some functions

The GdkEvent field is nullable on gtk_drag_begin*, and gtk_drag_cancel()
is a method, like the rest of the DnD API.

gtkdnd: Account for setting a same icon helper

g_set_object() will take care of ref'ing before destroying the previous
instance, which might actually be the same pointer.

https://bugzilla.gnome.org/show_bug.cgi?id=751401

Updated Portuguese translation

file chooser: Add a shortcut to focus the sidebar

Alt+P used to do this in the gtk2 filechooser (well, sometimes).
Add this back as a key binding.

https://bugzilla.gnome.org/show_bug.cgi?id=720684

place sidebar: Make removing bookmarks work again

This was accidentally broken in ae8c07c67cfa83d967 when I
forgot to switch the sense of a condition.

file chooser: Make delete confirmation match nautilus

No need to diverge here.

file chooser: Separate out delete and trash

We only ever show one of the two context menu items (and we prefer
Move to Trash over Delete). Only use the confirmation dialog when
deleting.

place sidebar: minor change

We generally prefer for loops to iterate over lists.

file chooser: Allow rename-to-self

When checking for a name clash for renaming, don't complain if the
name is unchanged. That's harmless.

wip: reshuffle name checks

file chooser: Remove unused field